Skip to content

Comments

SSO doc updates #24129

Open
akristen wants to merge 4 commits intodocker:mainfrom
akristen:sso-updates
Open

SSO doc updates #24129
akristen wants to merge 4 commits intodocker:mainfrom
akristen:sso-updates

Conversation

@akristen
Copy link
Contributor

@akristen akristen commented Feb 10, 2026

This PR updates SSO documentation to address inconsistencies, per tech support's jira ticket. I've made some additional changes as well:

  • I've removed 4/5ish call outs throughout the doc. Callouts lose effectiveness if you use them often.
  • Created visual distinction in the ToC with strategic h2s and h3s.
  • I've condensed certain redundant UI steps like "Save", "Finish", "Select X" and created more compound steps. This shortens the procedures and simplifies the ~weight of each substep.
  • I've updated Entra procedures where they've fallen out of sync

Work that needs to be done:

  • I've added some additional context to the Testing section, but it needs to be reworked generally.
  • I need to verify Azure Connect steps, including adding some context for steps that seem self-evident but could introduce uncertainty.
  • Idk what happened with the one callout but I need to fix it lol.
  • I want to add more context to some of the Azure Connect steps.
  • The ticket has some stuff for UI consistency across third party providers and our docs. Need to look for those mismatches more deeply.

While I personally don't prefer titles with "Step 1: jfdsfjsdf", I'm more than happy to defer if we think it'll help users understand the sequence as linear!! :) lmk

@netlify
Copy link

netlify bot commented Feb 10, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 73d8391
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/698bbf3cd7387d0008ff98ff
😎 Deploy Preview https://deploy-preview-24129--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@akristen akristen self-assigned this Feb 10, 2026
@akristen akristen added the area/admin Relates to Docker Admin label Feb 10, 2026
@akristen akristen marked this pull request as ready for review February 13, 2026 15:13
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation Review

I've reviewed the SSO documentation updates and found several issues that should be addressed:

Critical Issues (2)

  • Malformed callout syntax (line 119): The IMPORTANT callout has incorrect indentation that will break rendering
  • Stray character in callout (line 120): An embedded ">" character disrupts the text

Formatting Issues (2)

  • Inconsistent hyphenation (line 13): "set-up" should be "setup" per style guide
  • Extra space in list (line 30): Bullet point has two spaces instead of one

Please see the inline comments for details and suggested fixes.

Copy link

@gonzo919 gonzo919 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updates complete

{{< tab name="Okta SAML" >}}

1. Sign in to your Okta account and open the Admin portal.
1. Select **Administration** and then **Create App Integration**.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should read "Applications"

{{< tabs >}}
{{< tab name="Okta SAML" >}}

1. In Okta, select your app and go to **View SAML setup instructions**.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should say, In Okta, select your app, go to Sign On tab and go to View SAML setup instructions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/admin Relates to Docker Admin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants